Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Construction Pouches carry filled Sandbags + Bag Stack Change #477

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

GriffinMan4455
Copy link

@GriffinMan4455 GriffinMan4455 commented Sep 26, 2024

About the pull request

Simple change that lets filled sandbags be carried by conpouches, as well as ups their stack sizes to be more comparable to sheets.
Sandbags go up to 35 bags, with the large_stack spawnable having 50 simply for prop convenience

Did some quick and dirty testing and everything looks fine but this is also my first PR literally ever

Explain why it's good for the game

I made this change in order to encourage people actually using sandbags. As it stands now, nobody ever brings sandbags, and I think from both a gameplay and thematic/lore perspective it makes much more sense for Marines (or any combat force, really) to have sandbags on-hand for quick emergency fortification rather than hundreds of pounds of steel sheets.

As it stands now, metal sheets are just the default, required object to bring if the platoon things they will ever need fortifications and I find that weird honestly. It also doesn't make sense that sandbags are larger and take up more space than metal sheets, despite being slower since you could only carry empty bags, not being able to be carried in conpouches when filled, and enduring less damage than metal sheets anyways.

Testing Photographs and Procedure

Screenshots & Videos

image

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑

/:cl:

@GriffinMan4455
Copy link
Author

I am open to any and all feedback on the numbers changes, I honestly thought about upping the conpouch slot count as well, to 4, and then trying to find out how to restrict one of them to JUST the E-tool, but figured it would be fine as-is

@GriffinMan4455 GriffinMan4455 changed the title Conpouch change Construction Pouches carry filled Sandbags + Bag Stack Change Sep 26, 2024
code/game/objects/items/storage/pouch.dm Outdated Show resolved Hide resolved
code/game/objects/items/stacks/sandbags.dm Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants